Skip to content

Conversation

@ywangd
Copy link
Member

@ywangd ywangd commented Mar 25, 2025

This PR adds a minimal amount information about multi-project so that the existing distributed guide is up-to-date.

Relates: #123565
Resolves: ES-10985

This PR adds a minimal amount information about multi-project so that
the existing distributed guide is up-to-date.

Relates: elastic#123565
Resolves: ES-10985
@ywangd ywangd added >non-issue :Distributed Coordination/Task Management Issues for anything around the Tasks API - both persistent and node level. v9.1.0 labels Mar 25, 2025
@ywangd ywangd requested review from a team and pxsalehi March 25, 2025 06:55
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-distributed-coordination (Team:Distributed Coordination)

@elasticsearchmachine elasticsearchmachine added the Team:Distributed Coordination Meta label for Distributed Coordination team label Mar 25, 2025
Copy link
Member

@pxsalehi pxsalehi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (provided the one comment I have is addressed)


[ClusterState]:https://github.com/elastic/elasticsearch/blob/main/server/src/main/java/org/elasticsearch/cluster/ClusterState.java
[Metadata]:https://github.com/elastic/elasticsearch/blob/main/server/src/main/java/org/elasticsearch/cluster/metadata/Metadata.java
/elasticsearch/cluster/coordination/CoordinationMetadata.java
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this seems broken.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah you are right. It is a leftover that should have been removed. See 262432c
Thanks for catching it.

@ywangd ywangd added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Mar 26, 2025
@elasticsearchmachine elasticsearchmachine merged commit 6be0bf5 into elastic:main Mar 26, 2025
7 checks passed
@ywangd ywangd deleted the ES-10985-minor-update-to-distributed-guide branch March 26, 2025 01:48
omricohenn pushed a commit to omricohenn/elasticsearch that referenced this pull request Mar 28, 2025
…125556)

This PR adds a minimal amount information about multi-project so that
the existing distributed guide is up-to-date.

Relates: elastic#123565 Resolves:
ES-10985
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Distributed Coordination/Task Management Issues for anything around the Tasks API - both persistent and node level. >non-issue Team:Distributed Coordination Meta label for Distributed Coordination team v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants